Skip to content

2.26: Content Managers! #178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from
Open

2.26: Content Managers! #178

wants to merge 12 commits into from

Conversation

Borketh
Copy link
Member

@Borketh Borketh commented Feb 24, 2025

There were intermittent issues within Crashes where tasks' assigned files were closed before the tasks could execute. This solves that with a way overcomplicated synchronization system. Hopefully I thought of everything.

The content manager class can also be used to very simply get all files from a message, whether they're directly attached or linked.

There are also methods to make a group of File objects, which can be used to send multiple files along with a message.

Closes #177

@Borketh Borketh added the enhancement New feature or request label Feb 24, 2025
@Borketh Borketh requested a review from Feyko February 24, 2025 23:24
@Borketh Borketh self-assigned this Feb 24, 2025
Borketh and others added 6 commits February 28, 2025 15:23
this configuration appears to have no import issues, regardless of how and where the bot is started. this works for

poetry run fred
python -m fred
python fred/__main__.py
python fred

and the jetbrains profiler can launch it correctly (but due to the persistent nature of the bot, it does not accurately profile anything)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🆕 New
1 participant